Skip to content

Conversation

@MarkTripod-Duo
Copy link
Contributor

Description

Added start_idv(), cancel_idv() and get_idv_status() methods to admin.py

Motivation and Context

New methods were added to maintain parity with new Admin API endpoints and Duo feature functionality

How Has This Been Tested?

The changes were tested manually as well as via new test cases in tests/admin/test_idv.py

Types of Changes

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to change)

@ansharun
Copy link

ansharun commented Sep 9, 2025

Hey @MarkTripod-Duo , it seems like your PR partially covers the work described in our ticket for the current sprint.
I've tested these changes locally and everything seems to be working well.

Screen.Recording.2025-09-09.at.13.50.47.mov

Could we merge this PR, or is there some additional work that needs to be done?

@MarkTripod-Duo
Copy link
Contributor Author

Hi @ansharun

I originally submitted the PR after updating my internal client while validating the new IDV API endpoints. If these changes overlap with your other work, then this PR can be cancelled. Otherwise, I believe it would just need to be merged by someone with write access.

@jeffreyparker jeffreyparker merged commit f41eb47 into duosecurity:master Sep 10, 2025
5 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants